Skip to content

fix: adding rhel-9 compiler install and sec updates manual upload instructions #909

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Aug 8, 2025
Merged

fix: adding rhel-9 compiler install and sec updates manual upload instructions #909

merged 5 commits into from
Aug 8, 2025

Conversation

balakoteswar
Copy link
Contributor

Proposed changes

Adding the instructions for

  • RHEL-9 compiler install
  • Uploading the security updates manually from the NGINX repository

Testing

I have performed local testing.

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content. ↩

@balakoteswar balakoteswar requested a review from a team as a code owner July 30, 2025 14:13
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager labels Jul 30, 2025
@github-actions GitHub Actions
Copy link

✅ Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/909/

@balakoteswar balakoteswar self-assigned this Jul 30, 2025
mjang
mjang reviewed Jul 30, 2025
View reviewed changes
Copy link
Contributor

@mjang mjang left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current line 102 includes ### RHEL 8.1 or later. (This PR adds a section: ### RHEL 9 or later)

Can we replace with ### RHEL 8.x

Also, I'm unable to check the commands -- I trace this to an error in the NAP WAF docs, which still point to RHEL 7 (CentOS 7, actually)

Screenshot 2025-07-30 at 9 14 10 AM

@balakoteswar
Copy link
Contributor Author

Also, I'm unable to check the commands -- I trace this to an error in the NAP WAF docs, which still point to RHEL 7 (CentOS 7, actually)

Hi @mjang,
Could you please provide more details regarding this?

@mjang
Copy link
Contributor

mjang commented Jul 30, 2025

Also, I'm unable to check the commands -- I trace this to an error in the NAP WAF docs, which still point to RHEL 7 (CentOS 7, actually)

Hi @mjang, Could you please provide more details regarding this?

As shown from the second .repo file in the link, we point to CentOS 7. (I've tried a couple of variations, such as $releasever, but that didn't work.

baseurl=https://pkgs.nginx.com/app-protect-x-oss/centos/7/$basearch/

@mjang
Copy link
Contributor

mjang commented Jul 30, 2025

So I try various commands and end up with errors like:

$ sudo yum install app-protect-module-oss
Updating Subscription Management repositories.
nginx-app-protect repo                                                                               159  B/s | 153  B     00:00    
Errors during downloading metadata for repository 'app-protect-x-oss':
  - Status code: 403 for https://pkgs.nginx.com/app-protect-x-oss/centos/9/x86_64/repodata/repomd.xml (IP: 18.198.112.44)
Error: Failed to download metadata for repo 'app-protect-x-oss': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried

@balakoteswar
Copy link
Contributor Author

As shown from the second .repo file in the link, we point to CentOS 7. (I've tried a couple of variations, such as $releasever, but that didn't work.
baseurl=https://pkgs.nginx.com/app-protect-x-oss/centos/7/$basearch/

Hi @mjang,

Could you please clarify how your comments relate to this PR? It would be helpful if you could point out the exact location I should look into.

Also, could you explain what the second .repo file refers to?

Where in this PR are we referencing CentOS 7?

I couldn’t find any mention of baseurl=https://pkgs.nginx.com/app-protect-x-oss/centos/7/$basearch/. Could you point me to where it’s being used?

@balakoteswar
Copy link
Contributor Author

So I try various commands and end up with errors like:

$ sudo yum install app-protect-module-oss
Updating Subscription Management repositories.
nginx-app-protect repo                                                                               159  B/s | 153  B     00:00    
Errors during downloading metadata for repository 'app-protect-x-oss':
  - Status code: 403 for https://pkgs.nginx.com/app-protect-x-oss/centos/9/x86_64/repodata/repomd.xml (IP: 18.198.112.44)
Error: Failed to download metadata for repo 'app-protect-x-oss': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried

Also, how is this comment related to this PR? Could you please provide some details?

JTorreG
JTorreG approved these changes Jul 31, 2025
View reviewed changes
Copy link
Contributor

@JTorreG JTorreG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammar and style lgtm

travisamartin
travisamartin reviewed Aug 7, 2025
View reviewed changes
travisamartin
travisamartin reviewed Aug 7, 2025
View reviewed changes
@balakoteswar balakoteswar merged commit 98340aa into main Aug 8, 2025
8 checks passed
@balakoteswar balakoteswar deleted the rhel-9-steps branch August 8, 2025 14:26
salonichf5 pushed a commit to salonichf5/documentation that referenced this pull request Aug 11, 2025
@balakoteswar @travisamartin @salonichf5
fix: adding rhel-9 compiler install and sec updates manual upload ins…
aa3ab34 
…tructions (nginx#909)

* fix: adding rhel-9 compiler install and sec updates manual upload instructions

* fix: address review comments

* fix: Update content/nim/nginx-app-protect/setup-waf-config-management.md

Co-authored-by: Travis Martin <[email protected]>

* fix: Update content/nim/nginx-app-protect/setup-waf-config-management.md

Co-authored-by: Travis Martin <[email protected]>

---------

Co-authored-by: Travis Martin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mjang mjang mjang left review comments

@travisamartin travisamartin travisamartin approved these changes

@JTorreG JTorreG JTorreG approved these changes

@mkingst mkingst Awaiting requested review from mkingst

Assignees

@balakoteswar balakoteswar

Labels
documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@balakoteswar @mjang @travisamartin @JTorreG